feat(scout-url): parametro --user-agent configurabile#68
Conversation
matteocavo
left a comment
matteocavo
left a comment
There was a problem hiding this comment.
Nota minore, non bloccante:
non c’è ancora un test che verifichi esplicitamente il passaggio di un --user-agent custom fino a requests.get, quindi la nuova feature ha copertura indiretta ma non diretta.
Verdetto:
nessun finding HIGH/CRITICAL
approvabile
|
Implementato il test unitario richiesto da @matteocavo. Il nuovo test est_probe_url_passes_custom_user_agent verifica che il parametro user_agent arrivi correttamente a |
matteocavo
left a comment
matteocavo
left a comment
There was a problem hiding this comment.
Non ho trovato blocker su toolkit#68.
Verifica fatta:
diff coerente: il nuovo --user-agent passa dalla CLI a probe_url() senza cambiare il default
test mirati verdi:
.\.venv\Scripts\python.exe -m pytest tests\test_cli_scout_url.py -q
esito: 4 passed
In più, ora c’è anche il test giusto sul nuovo comportamento:
test_probe_url_passes_custom_user_agent in test_cli_scout_url.py
Verdetto:
nessun HIGH o CRITICAL
approvabile
2 participants
Cosa cambia
Aggiunge il parametro
--user-agental comandotoolkit scout-url.Diversi portali istituzionali italiani (ACI, MEF DAG, Ministero Salute) bloccano richieste con User-Agent non standard restituendo 403 o 404 fittizi, rendendo impossibile verificare accesso e formato senza aprire il browser manualmente.
Modifica
probe_url()accetta orauser_agentcome parametro (default invariato:dataciviclab-toolkit/scout-url)--user-agentcon lo stesso default — nessuna breaking changeTest
4 test esistenti passano senza modifiche.
Closes #67